Skip to content

Conversation

trivikr
Copy link
Contributor

@trivikr trivikr commented Sep 30, 2025

Issue #, if available:
https://docs.gradle.org/current/userguide/upgrading_major_version_9.html

Description of changes:
Bumps Gradle to 9.0.0


By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@trivikr trivikr force-pushed the gradle-9 branch 2 times, most recently from b89ffa3 to 055180e Compare September 30, 2025 15:30
@trivikr trivikr marked this pull request as ready for review September 30, 2025 18:17
@trivikr trivikr requested a review from a team as a code owner September 30, 2025 18:17
@trivikr
Copy link
Contributor Author

trivikr commented Sep 30, 2025

The Java code was updated because of checkstyle errors

Error: eckstyle] [ERROR] /home/runner/work/smithy-typescript/smithy-typescript/smithy-typescript-codegen/src/main/java/software/amazon/smithy/typescript/codegen/documentation/StructureExampleGenerator.java:297:5: Class ShapeTracker should be declared as final. [FinalClass]
CommandGeneratorTest > writesOperationContextParamValues() PASSED
Error: eckstyle] [ERROR] /home/runner/work/smithy-typescript/smithy-typescript/smithy-typescript-codegen/src/main/java/software/amazon/smithy/typescript/codegen/auth/http/HttpAuthOptionProperty.java:25:8: Redundant 'final' modifier. [RedundantModifier]
Error: eckstyle] [ERROR] /home/runner/work/smithy-typescript/smithy-typescript/smithy-typescript-codegen/src/main/java/software/amazon/smithy/typescript/codegen/auth/http/HttpAuthOptionProperty.java:44:12: Redundant 'static' modifier. [RedundantModifier]
Error: eckstyle] [ERROR] /home/runner/work/smithy-typescript/smithy-typescript/smithy-typescript-codegen/src/main/java/software/amazon/smithy/typescript/codegen/auth/http/HttpAuthOptionProperty.java:44:19: Redundant 'final' modifier. [RedundantModifier]
Error: eckstyle] [ERROR] /home/runner/work/smithy-typescript/smithy-typescript/smithy-typescript-codegen/src/main/java/software/amazon/smithy/typescript/codegen/auth/http/ResolveConfigFunction.java:20:8: Redundant 'final' modifier. [RedundantModifier]
Error: eckstyle] [ERROR] /home/runner/work/smithy-typescript/smithy-typescript/smithy-typescript-codegen/src/main/java/software/amazon/smithy/typescript/codegen/auth/http/HttpAuthSchemeParameter.java:24:8: Redundant 'final' modifier. [RedundantModifier]
Error: eckstyle] [ERROR] /home/runner/work/smithy-typescript/smithy-typescript/smithy-typescript-codegen/src/main/java/software/amazon/smithy/typescript/codegen/auth/http/ConfigField.java:31:8: Redundant 'final' modifier. [RedundantModifier]

Logs: https://github.com/smithy-lang/smithy-typescript/actions/runs/18134806992/job/51610642108?pr=1717#step:5:84

@trivikr trivikr marked this pull request as draft September 30, 2025 18:50
@trivikr
Copy link
Contributor Author

trivikr commented Sep 30, 2025

This PR will be merged after codegen 0.36.0 release in #1719

@trivikr trivikr marked this pull request as ready for review September 30, 2025 21:13
@trivikr trivikr merged commit ceb0a0d into main Sep 30, 2025
13 checks passed
@trivikr trivikr deleted the gradle-9 branch September 30, 2025 22:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants